Feat/color grading - #334
Conversation
Lillian-ran
left a comment
Lillian-ran
left a comment
There was a problem hiding this comment.
Hi Deezper, I noticed there are unresolved merge conflict markers (<<<<<<<, =======, >>>>>>>) left in this file (MainActivity.java).
Please resolve the conflicts locally before pushing again: Edit the file to manually keep the correct code and remove the conflict markers, then commit and push the cleaned file.
|
Hi Yanran, Ive done the changes now |
|
Hi Deezper, thanks for updating the PR. I tested the changes but currently I’m unable to log into the app — after entering account and password, it just loops back to the login page. This seems to be related to the changes in Also, for clarity and easier testing, I suggest adding a short demo video of your feature working. That will help us quickly verify the functionality. Thanks! |
|
Hi Yanran, Thanks for the feedback i have revieved and done the necessary changes and below attached is the demo video https://drive.google.com/file/d/1QbuNMGBMAEXfIIHmg4jhMrCRrBnbKpCV/view?usp=sharing |
|
Feature Documentation - https://docs.google.com/document/d/1bT-6UrX_6KiG5NLqP6B1U4YNZo8goGJc9b22PfCd6g4/edit?usp=sharing |
2 participants
Replace score with count-based bands
Thresholds: LOW 0–14, MEDIUM 15–59, HIGH 60+
Theme updates via ThreatThemeManager (cards recolor automatically)
Updated: DatabaseThreatRepository, ThreatRepository, MainViewModel, MainActivity, drawables
Boundary checks: 14→15 and 59→60